Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 1Password support. #175

Merged
merged 1 commit into from
Jan 12, 2014
Merged

Add 1Password support. #175

merged 1 commit into from
Jan 12, 2014

Conversation

shrwnsan
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 2a15fcb on stechico:release-1password into b2c16c0 on lra:master.

@lra
Copy link
Owner

lra commented Jan 12, 2014

Not sure I understand here, I'm a user of 1Password, and it already syncs well with Dropbox.
Also I don't have some of the files you specify, e.g. APP_SUPPORT + '1Password/License'
Can you elaborate on why we need this in ?

@lra lra merged commit 2a15fcb into lra:master Jan 12, 2014
@shrwnsan
Copy link
Contributor Author

Hi @lra. I only saw the option in the 1Password (v3.x) Settings>General>Data File to sync "1Password.agilekeychain". It didn't sync the rest. I guess if we're just wanting "settings" as a whole with Mackup, then the rest wouldn't matter. Though, I'd would probably want the License on there to sync up.

screen shot 2014-01-13 at 7 08 15 am

@vitorgalvao
Copy link
Contributor

These files pertain to users that bought 1Password directly from Agile, and not from the Mac App Store. This might pose a problem, since if support for MAS apps is ever added, there might very well be conflicts.

Like with Google Chrome, the default sync is incomplete (what 1Password syncs is the keychain, not the settings).

@shrwnsan
Copy link
Contributor Author

Thanks for the insight @vitorgalvao

@shrwnsan shrwnsan deleted the release-1password branch January 14, 2014 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants